Skip to content

Conversation

@raethlo
Copy link
Contributor

@raethlo raethlo commented Oct 30, 2025

This PR adds docs for a new platfrom feature: Actor permissions.

tobice
tobice previously requested changes Oct 31, 2025
Copy link
Contributor

@tobice tobice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gave it a quick look; it still needs a bit of love. I'd even rethink the structure a bit ;D

- `["READ"]` — the Actor may read from the referenced resource(s).
- `["READ", "WRITE"]` — the Actor may read and write to the referenced resource(s).

Notes:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's mention that this will be communicated to the users via the form (the tooltip).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the copy that needs to be updated. Let's not forget about updating this screenshot as well.

@apify-service-account
Copy link

Preview for this PR was built for commit 5b8c483a and is ready at https://pr-2053.preview.docs.apify.com!

@raethlo raethlo marked this pull request as ready for review November 5, 2025 17:26
@raethlo raethlo requested review from tobice and removed request for fnesveda and janbuchar November 5, 2025 17:27
@apify-service-account
Copy link

Preview for this PR was built for commit f0e0ded7 and is ready at https://pr-2053.preview.docs.apify.com!

@raethlo raethlo requested a review from nmanerikar November 6, 2025 09:25
Copy link

@nmanerikar nmanerikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments, suggestions and corrections.

@apify-service-account
Copy link

Preview for this PR was built for commit 7413720b and is ready at https://pr-2053.preview.docs.apify.com!

Copy link

@nmanerikar nmanerikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One final comment, and reminder to update the screenshots.
Other than that, this looks fine.
I assume there will be a review from the content team as well?


![Full-permissions badge shown on Actor detail page](../development/permissions/images/end_user_ux_full_permissions.png)

Over time, the distinction between these permission levels will become more prominent. For example, Actors requiring full permissions may display an orange badge, appear lower in Store rankings, or show extra confirmation dialogs when you run them. Whenever possible, choose Actors that use **limited permissions**. They are safer, easier to trust, and sufficient for most workflows.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I missed this previously, but in keeping with @TC-MO's comment about dropping mentions of future changes, we should drop this part.

@apify-service-account
Copy link

Preview for this PR was built for commit 75c6ff81 and is ready at https://pr-2053.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit dda406ef and is ready at https://pr-2053.preview.docs.apify.com!

Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of the changes are for conciness or logic flow, few comments with questions

raethlo and others added 4 commits November 12, 2025 12:48
First batch of wording, flow and consistency edits

Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
@apify-service-account
Copy link

Preview for this PR was built for commit da41a8aa and is ready at https://pr-2053.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit de855a46 and is ready at https://pr-2053.preview.docs.apify.com!

Another batch of copy/flow

Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
@apify-service-account
Copy link

Preview for this PR was built for commit 3c3c4771 and is ready at https://pr-2053.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 2649905f and is ready at https://pr-2053.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit eae6f7fd and is ready at https://pr-2053.preview.docs.apify.com!

Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Final nits, once this all these comments are addressed we are ready for merge

raethlo and others added 2 commits November 13, 2025 10:44
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
@apify-service-account
Copy link

Preview for this PR was built for commit 58b0cacb and is ready at https://pr-2053.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit b59f7ee4 and is ready at https://pr-2053.preview.docs.apify.com!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants